-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update pulumi-terraform-bridge to v3.33.0 and pulumi/sdk to 3.44.2 #1161
Conversation
Does the PR have any schema changes?Looking good! No breaking changes found. |
For this branch:
For master:
So this is a step backwards in terms of coverage. Probably worth spending a little time going through individual errors to understand why they're failing. |
Trying to narrow down the offending change in a crude binary search, I upgraded to versions somewhat in the middle between current and latest.
This preserves the example, so the change came later. |
|
Confirmed via local experimentation that using a recent master of pulumi/pulumi brings the example conversion failures back down, thanks to pulumi/pulumi#11311. We need to upgrade terraform-bridge to use a version of pulumi that includes this fix. |
cc: @viveklak I think you mentioned you were investigating how to get the bridge upgrade unblocked. Let us know if there's anything we can do to aid the roll-out when it gets there |
closing in favor of a new PR with more recent versions |
The bulk of the changes is from pulumi/pulumi-terraform-bridge#612.
The PR is still a draft because I'd like to take a closer look at why many examples disappeared.